Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

Add a script to copy eval run results to Turso#2452

Merged
cte merged 1 commit into
mainfrom
cte/copy-run-script
Apr 10, 2025
Merged

Add a script to copy eval run results to Turso#2452
cte merged 1 commit into
mainfrom
cte/copy-run-script

Conversation

@cte

@cte cte commented Apr 10, 2025 •
edited by ellipsis-dev Bot
Loading

Copy link
Copy Markdown
Contributor

Context

Implementation

Screenshots

before after

How to Test

Get in Touch

Important

Add copy-run.mts script to copy eval run results to Turso, with necessary configuration updates.

  • Script Addition:
    • Adds copy-run.mts script to copy eval run results to Turso.
    • Handles copying of run and task metrics, with error handling for missing data and insertion failures.
  • Configuration:
    • Adds db:copy-run script command in package.json.
    • Updates turbo.json to include TURSO_CONNECTION_URL and TURSO_AUTH_TOKEN in globalEnv.

This description was created by Ellipsis for 92067df. It will automatically update as commits are pushed.

@cte cte requested a review from mrubens as a code owner April 10, 2025 04:16
@changeset-bot

changeset-bot Bot commented Apr 10, 2025

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 92067df

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Apr 10, 2025
@cte cte merged commit 4e71626 into main Apr 10, 2025
@cte cte deleted the cte/copy-run-script branch April 10, 2025 04:18
@github-project-automation github-project-automation Bot moved this from New to Done in Roo Code Roadmap Apr 10, 2025
@dosubot dosubot Bot added the Enhancement New feature or request label Apr 10, 2025
ellipsis-dev[bot]
ellipsis-dev Bot reviewed Apr 10, 2025
View reviewed changes
Comment thread evals/packages/db/scripts/copy-run.mts
const copyRun = async (runId: number) => {
const destDb = drizzle({
schema,
connection: { url: process.env.TURSO_CONNECTION_URL!, authToken: process.env.TURSO_AUTH_TOKEN! },

@ellipsis-dev ellipsis-dev Bot Apr 10, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider checking that TURSO_CONNECTION_URL and TURSO_AUTH_TOKEN are defined instead of using non-null assertions. This improves error clarity if the env variables are missing.

Comment thread evals/packages/db/scripts/copy-run.mts

console.log(`Copying ${tasks.length} tasks`)

for (const task of tasks) {

@ellipsis-dev ellipsis-dev Bot Apr 10, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The loop inserting each task and its taskMetrics might benefit from a transaction to ensure atomicity and data consistency. Consider wrapping these DB operations in a transaction.

ouob-tw added a commit to ouob-tw/BigFont-Roo-Code that referenced this pull request May 6, 2025
@ouob-tw @github-actions @mrubens
Merge from roo code 3.11.12
d953fed 
fix: update ChatRowContent(user msg in chat windows) background and font size for better styling

Add o1-pro to api.ts (RooCodeInc#2433)

Add the o1-pro model to the openai section. Sourced model info from: https://platform.openai.com/docs/models/o1-pro

Update contributors list (RooCodeInc#2411)

docs: update contributors list [skip ci]

Co-authored-by: mrubens <mrubens@users.noreply.github.com>

Add custom instructions for de (RooCodeInc#2383)

v3.11.11 (RooCodeInc#2435)

API fixes (RooCodeInc#2438)

Changeset version bump (RooCodeInc#2436)

* changeset version bump

* Update CHANGELOG.md

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>

More sane evals default concurrency + staggered startup (RooCodeInc#2441)

Fix gh fork command (RooCodeInc#2442)

Add a script to copy eval run results to Turso (RooCodeInc#2452)

Fall back on aggressive line number stripping in diffs (RooCodeInc#2453)

* Add option for aggressive line number stripping

* Fall back on aggressive line number stripping in diffs

Make Grok3 streaming work with OpenAI Compatible (RooCodeInc#2449)

Update contributors list (RooCodeInc#2434)

docs: update contributors list [skip ci]

Co-authored-by: mrubens <mrubens@users.noreply.github.com>

v3.11.12 (RooCodeInc#2454)

Changeset version bump (RooCodeInc#2455)

* changeset version bump

* Updating CHANGELOG.md format

* Update CHANGELOG.md

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: R00-B0T <github-actions@github.com>
Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@mrubens mrubens Awaiting requested review from mrubens mrubens is a code owner

1 more reviewer

@ellipsis-dev ellipsis-dev[bot] ellipsis-dev[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Enhancement New feature or request size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@cte